Skip to content

Run validation suite on multiple versions and store them separately in folders#71

Merged
hahnjo merged 4 commits into
root-project:mainfrom
undefined-panda:multiple-versions
Jun 16, 2026
Merged

Run validation suite on multiple versions and store them separately in folders#71
hahnjo merged 4 commits into
root-project:mainfrom
undefined-panda:multiple-versions

Conversation

@undefined-panda

Copy link
Copy Markdown
Contributor

The validation suite can now be run by specifying folders for dictionaries, ROOT binary files and JSON files created accordingly. That way the .root files generated by version X can be read by other versions to create .json files.

@undefined-panda undefined-panda deleted the multiple-versions branch June 11, 2026 15:02
@undefined-panda undefined-panda restored the multiple-versions branch June 12, 2026 07:11

@hahnjo hahnjo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe consider squashing commits 2,3,4 because only together the changes fully move dictionaries to a subdirectory. (there's also make store and move_files which is later removed again, but getting rid of that one would be an added bonus)

Comment thread types/map/nested/Makefile Outdated
Adjusted main Makefile to store all generated files in specified folders.
@undefined-panda undefined-panda force-pushed the multiple-versions branch 2 times, most recently from a70716e to 4698ea0 Compare June 16, 2026 09:42

@hahnjo hahnjo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, LGTM!

…files and JSON files are now written directly in designated folders.

DICT_DIR, WRITE_DIR and READ_DIR always end with '/', so that inner Makefiles can be run individually.

Adjusted DICT_DIR, WRITE_DIR and READ_DIR to always end with '/', so that inner Makefiles can be run individually
@hahnjo hahnjo merged commit 846b978 into root-project:main Jun 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants